Use CHECKED instead of ACTIVE for expanded expanders

https://bugzilla.gnome.org/show_bug.cgi?id=733967

configure: Move AC_USE_SYSTEM_EXTENSIONS up

We don't want to get warnings from autoconf about
AC_USE_SYSTEM_EXTENSIONS coming after AC_COMPILE_IFELSE.

quartz-dnd: make sure to set source window

https://bugzilla.gnome.org/show_bug.cgi?id=658722

Adwaita: no edge hilight for popover checks/radios

reftests: Fix Makefile

with non-installed tests the build would get an empty $(reftestdir)
which would screw up the LDFLAGS.

An rpath seems to be required to make libtool build a shared object.
Without an rpath line, it only builds a static object.

https://bugzilla.gnome.org/show_bug.cgi?id=735401

gtk-demo: Don't use mad mime type guessing

... for displaying resources. Instead use the proven and way more
reliable method of trial and error.

It's less code and more portable for a start.
But most of all it displays PNM files as text if you fail to compile
the gdk-pixbuf loader for it.

modifierstyle: Remove the changed signal

It's been unused since last commit

widget: Don't connect to GtkModifierStyle::changed

Changes to the modifier style will be picked up by the
GtkStyleProviderPrivate::changed signal. And that signal will actually
invalidate properly.

wayland: Ensure the parent is committed after repositioning subsurfaces

Subsurface position is deemed part of the state of the parent surface, so
ensure wl_surface_commit() happens on the parent surface if none is
scheduled, so the repositioning takes place.

treeview: Reset multipress controller after 2nd click

The pre-gestures code used to compare the current button press with the
previous one on !activate_on_single_click, and unset the previous event
data so ::row-activated would be emitted every 2 clicks.

So do the same with the multipress gesture and reset it after every 2nd
click to have ::row-activated emitted multiple times while manic clicking.

https://bugzilla.gnome.org/show_bug.cgi?id=735353

threads: Do not release the GDK lock if it hasn't been acquired yet

Since GLib ≥ 2.41, attempting to release an unlocked mutex will abort(),
as it happens on most systems already.

Given the lack of proper documentation on how to use GDK with threads,
there is code in the wild that does:

    gdk_threads_init ();
    gdk_init ();

    ...

    gtk_main ();

instead of the idiomatically correct:

    gdk_threads_init ();
    gdk_threads_enter ();

    gtk_init ();

    ...

    gtk_main ();

    ...

    gdk_threads_leave ();

Which means that gtk_main() will try to release the GDK lock, and thus
trigger an error from GLib.

we cannot really fix all the wrong code everywhere, and since it does
not cost us anything, we can work around the issue inside GDK itself, by
trying to acquire the GDK lock inside gdk_threads_leave() with
trylock().

https://bugzilla.gnome.org/show_bug.cgi?id=735428

build: make reftests work without installed tests

The libreftest.so module needs to be built in all cases.
Without installed tests, it needs to not be installed though.

https://bugzilla.gnome.org/show_bug.cgi?id=735401

Cleanup invalid comments (duplication of words)

https://bugzilla.gnome.org/show_bug.cgi?id=735301

Signed-off-by: Ikey Doherty <michael.i.doherty@intel.com>

button: Track gesture point to maintain priv->in_button

This makes the active state work invariably with both mouse/touch, and
regardless of X11 pointer emulation being friendly and sending crossing
events for the emulated pointer events in the latter.

This makes GtkButtons' active state look correct when pressing on
touchscreens on wayland.

https://bugzilla.gnome.org/show_bug.cgi?id=731380

entrycompletion: Mark popup window as subsurface on wayland

This will make the completion window use GDK_WINDOW_SUBSURFACE and be
shown relative to the parent window without involving compositor grabs.

https://bugzilla.gnome.org/show_bug.cgi?id=695504

gtkwindow: Add private API to mark as subsurface

This is a rather hackish way to let GTK+ widgets declare popup windows
as subsurfaces, so they may work on wayland without the need of xdg_popup,
and without many changes yet on the GTK+ side.

https://bugzilla.gnome.org/show_bug.cgi?id=695504

Adwaita: menu checks and radios making sense

Adwaita: set a background for GtkTextView

Needed for gedit line numbers, overridden by .view {} normally

Adwaita: minor fixes to GtkLabel

Adwaita: take rtl into consideration for GtkAssistant

Adwaita: GtkAssistant tweakings

Adwaita: treeview checks/radios minor fix

Superfluous icon shadow in checked hover state removed.

Adwaita: in app notification fixes

scalebutton: Handle smooth scroll events

This has the double effect to make scale button truly smooth, and make
scrolling work on wayland, since all scrolling is "smooth" there.

https://bugzilla.gnome.org/show_bug.cgi?id=720269

wayland: Fix x/y coordinate arguments on wl_data_device events

Those are wl_fixed_t, not int.

https://bugzilla.gnome.org/show_bug.cgi?id=697855

wayland: ensure gdk_device_manager_get_client_pointer() returns a master pointer

The internal ordering of devices isn't quite guaranteed, so ensure we're returning
the right one.

https://bugzilla.gnome.org/show_bug.cgi?id=697855

Revert "Adwaita: notebook background"

Not a good solution to the problem at hand (gedit line numbering)

This reverts commit fde03c1f8174a16120264eb689415b6f7535fc81.

wayland: unset sequence if the xdg surface is moved/resized through touch

The latest implicit grab serial is used in order to start the compositor
grab, If it belongs to a touch event, remove that touch sequence, as the
rest of the sequence will be gone for good.

This avoids stale sequences (and implicit grab info) after a window is
moved/resized.

https://bugzilla.gnome.org/show_bug.cgi?id=731380

wayland: Add internal API to unset a touch implicit grab

This removes both the wayland specific accounting, and the Gdk implicit
grab tracking.

https://bugzilla.gnome.org/show_bug.cgi?id=731380

wayland: Update the display serial on touch down/up events

https://bugzilla.gnome.org/show_bug.cgi?id=734374

wayland: Improve implicit grab serial retrieval

_gdk_wayland_device_get_button_press_serial() has been replaced by
_gdk_wayland_device_get_implicit_grab_serial(), which takes a touch/pointer
event and figures out the relevant serial, and
_gdk_wayland_device_get_last_implicit_grab_serial() which returns
the most recent serial.

The button press serial was currently used when operating popping up
xdg_shell/surface popups and window menus, so this is now touch aware, of
some sort.

https://bugzilla.gnome.org/show_bug.cgi?id=734374

wayland: Store per-touch touch_down serial

https://bugzilla.gnome.org/show_bug.cgi?id=734374

wayland: rename internal touch management functions

These functions arent quite related to the device manager, so rename these
more conveniently.

https://bugzilla.gnome.org/show_bug.cgi?id=734374

wayland: Protect against invalid keymaps gotten from the compositor

If the compositor sends a keymap that fails on "compilation",
xkb_keymap_new_from_string() returns NULL, which makes xkb_state_new()
crash when assuming there is a keymap.

In these cases, gdk must remain with a xkb_state to handle modifiers/keys
properly, so warn about the invalid keymap string, and keep the previous
keymap (currently initialized to "us")

https://bugzilla.gnome.org/show_bug.cgi?id=735389

wayland: create a wl_subsurface interface for GDK_WINDOW_SUBSURFACE windows

This subsurface is currently dependent on the transient_for parent, so the
subsurface is repositioned relative to it.

https://bugzilla.gnome.org/show_bug.cgi?id=729215

wayland: Acquire wl_subcompositor interface

This will be needed for GDK_WINDOW_SUBSURFACE windows.

https://bugzilla.gnome.org/show_bug.cgi?id=729215

gdk: Add GDK_WINDOW_SUBSURFACE window type

This window type can only be used on wayland so far, so NULL is returned
if it's attempted to be used on any other windowing backend.

https://bugzilla.gnome.org/show_bug.cgi?id=729215

wayland: Make toplevels' X/Y coordinates be 0

To all effects each window has its own "root" coordinates system, so set
toplevels at 0,0 in that coordinate system, so root coordinate calculations
are locally right.

https://bugzilla.gnome.org/show_bug.cgi?id=729215

Adwaita: notebook background

TextView: clear the background using the default bg color

Try to tidy up how the background is set on the textview:
 *) the .view class should be applied only to the text window, not
    to the margins
 *) when setting the background on the margins we must use .left etc
 *) use context_set_bg instead of manually setting the color

https://bugzilla.gnome.org/show_bug.cgi?id=735368

GtkTreeView: Avoid a crash with empty trees

The gesture code was assuming that the rb tree exists, which
will not be the case if the treeview is empty.

https://bugzilla.gnome.org/show_bug.cgi?id=735300

Adwaita: define styling for location-bar toolbars

- same inset style as searchbars

Adwaita: legible labels in headerbars in selection mode

Adwaita: selection-mode checked togglebuttons

- exposed in gnome-contacts for example.

Adwaita: selected text in GtkLabel

https://bugzilla.gnome.org/show_bug.cgi?id=735310

inspector: fix a crash in the CSS editor

gtk_css_section_get_end_position() can return a position one byte beyond
the end of the last line.

gtk_text_buffer_get_iter_at_line_index() accepts only valid
line_number/byte_index pairs. Another solution is to make the
GtkTextBuffer function less strict, by returning a boolean if the exact
position was found.

The CSS parser should also be fixed to always return valid positions.
But it's better to have a safety net in the CSS editor, just in case (a
warning could be print).

https://bugzilla.gnome.org/show_bug.cgi?id=735337

gdkwindow-quartz: Support native fullscreen mode

This implements native fullscreen mode on OS X >= 10.7. This patch
adds tracking of the fullscreen mode if externally modified, as well
as toggling the native fullscreen mode of the window as needed.

Ref the popover before activating the item

Otherwise the item could cause the popover to be destroyed,
for example a Quit item, and cause a warning to be printed.

GtkAboutDialog: Drop visibility-notify-event handler

The signal is deprecated, and the handler is only helping
for a really tiny corner case anyway.

https://bugzilla.gnome.org/show_bug.cgi?id=734961

Change gdk-pixbuf minimum required version to 2.30.0

https://bugzilla.gnome.org/show_bug.cgi?id=735253

Signed-off-by: Ikey Doherty <michael.i.doherty@intel.com>

Some release notes for 3.14

Adwaita: use osd style for in-app notifications

xdg-shell: Bump version

Adwaita: added a workaround to make sass behave

GtkColorChooser: Make color plane react to mouse again

Fallout from changing the defaults for gesture properties.

Adwaita: dark active buttons for osd

Adwaita: @define-color "compatible" with sass 3.4.0

wayland: Add defines to convert between event sequences and touch slots

This makes it clearer in code what's going on there.
GDK_SLOT_TO_EVENT_SEQUENCE isn't used yet, but will be useful in the
future.

textview: Unset bubble timeout ID right after requesting the clipboard

The timeout ID used to be unset after we got the targets from the
clipboard, but there's still a moment between the clipboard request and
the GDK_SELECTION_NOTIFY event that the ID points to an already gone
timeout.

x11: Fix possible leak of GdkEvents

If the filter didn't return GDK_EVENT_TRANSLATE, the event struct would be
left unused and leaked.

Adwaita: kill a bogus background-color

Adwaita: first round at osd revamp

...buttons, gtkscale and toolbars, more to follow...

GtkFileChooserWidget: Use ratio in formatted times

This matches what we do in other places now.

Avoid picking up long description as translator comment

Rearrange a few lines to help xgettext avoid picking up the
long description of GtkFileChooserButton as translator comment.

Avoid picking up long description as translator comment

Rearrange a few lines to help xgettext avoid picking up the
long description of GtkAboutDialog as translator comment.

Remove an extraneous translatable string

Adwaita: Insensitive labels

There is an insensitive label in widget-factory which was
not getting any different from the sensitive label next
to it. With this patch, it does, again.

Adwaita: Add assistant sidebar theming

This is just enough to draw the sidebar background and
markt he current page as active.

GtkAssistant: Don't use a draw cb for the sidebar

We can do this with css nowadays.

gtk: Fix return-annotation of gtk_get_default_language to "transfer none"

GtkPageSetup: Improve formatting of size

Use U+00D7 MULTIPLICATION SIGN and U+200A HAIR SPACE instead of plain
old 'x' and ' ', following https://wiki.gnome.org/Design/OS/Typography.

Sadly, many fonts don't have space variations, so Pango/harfbuzz fall
back to using the regular space glyph anyway.

Add tests for parsing int64/uin64 in GtkBuilder

This tests the fix in the previous commit.

GtkBuilder: Support parsing int64/uin64 values

This was missing; the code to parse these numbers was there,
but the switch didn't have cases for these types.

https://bugzilla.gnome.org/show_bug.cgi?id=602443

Include config.h first

Otherwise, we miss the _GNU_SOURCE #define that configure
puts there.

Simplify _gtk_text_buffer_get_line_log_attrs()

NULL was returned in case of an empty last line. Every users needed to
special-case this. Now it will return the expected result: char_len of 0
with one PangoLogAttr.

In compute_log_attrs(), 'paragraph' will be the empty string "" with
'char_len' == 0.
pango_get_log_attrs() works fine with an empty string, it will return
one correct PangoLogAttr (because there is one text position for the
empty string).

It fixes the unit tests for gtk_text_iter_is_cursor_position().

https://bugzilla.gnome.org/show_bug.cgi?id=156164

textbuffer: unit tests for the empty last line

For functions using _gtk_text_buffer_get_line_log_attrs():
- gtk_text_buffer_backspace()
- some gtk_text_iter functions (word/sentence/cursor boundaries)

As the FIXME comments show, there is a bug with
gtk_text_iter_is_cursor_position() for an empty last line.

https://bugzilla.gnome.org/show_bug.cgi?id=156164

GtkPopover: Fix rtl positioning

There was some confusion between unflipped and flipped positions.
Both final_position and current_position are meant to be unflipped,
and get_effective_position() needs to be applied to them to get
a flipped position. _gtk_window_set_popover_position() also expects
an unflipped position.

https://bugzilla.gnome.org/show_bug.cgi?id=735014

Adwaita: try to be nicer on performances with entry shadows

Updated POTFILES.skip

Adwaita: no edge highlight for backdrop check/radios

Adwaita: remove edge highlight form the assets

- using icon-shadow now, allows to treat special cases
  without introducing assets

Updated Traditional Chinese translation(Hong Kong and Taiwan)

Adwaita: some separators simplification

Accidentally fixing the backdrop separators in sidebars turning
black

Adwaita: inline toolbar buttons fixes

Properly styled single buttons in inline toolbars and fixed
a superflous border chiming in in backdrop.

Deprecate GTK_CHECK_BACKEND in favour of PKG_CHECK_MODULES

This is easy to do with pkg-config too.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=641638
Bug-Debian: https://bugs.debian.org/756476
Reviewed-by: Matthias Clasen

Deprecate AM_PATH_GTK_3_0 in favour of PKG_CHECK_MODULES

That's what GNOME applications use.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=641638
Bug-Debian: https://bugs.debian.org/756476
Reviewed-by: Matthias Clasen

Use AC_USE_SYSTEM_EXTENSIONS to get _GNU_SOURCE, _XOPEN_SOURCE etc.

Similar to Bug #684123 in GLib.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=641638
Bug-Debian: https://bugs.debian.org/756476
Reviewed-by: Matthias Clasen

css: Round shadow extents properly

Otherwise drawing will be clipped.

Testcase included

roundedbox: Only grow border-radius if there is one

This is relevant for shadow spread and goes in line with the web.

Testcase included.

label: Respect box-shadow when computing clip

Testclase included.

Also reorder headers to be alphabetic.

widget: Make _gtk_set_simple_clip() take an optional content clip

stylecontext: Remove GTK_CSS_CHANGE_FORCE_INVALIDATE

This was introduced as a hackish way in 3.6 to make font updates
propagate properly. But since then, font handling has been changed and
this flag is no longer necessary.

stylecontext: Use right invalidation func

gtk_style_context_invalidate_internal() will respect only the current
saved state of the style context, which is wrong when updating the scale.
In that case, the whole style context needs updating.

GtkTextView: various code clean-ups

- only one blank line is enough to separate code sections.
- the 'signals' variable was in the middle of function prototypes.
- compare pointers to NULL in some conditions ("if(blah) should be used
  only if blah is a boolean variable). It makes the code clearer.
- various other things.

tests textbuffer: don't use GdkColor (deprecated)

Mark the GdkColor struct as deprecated

gtk: Fix transfer annotations gtk_binding_set_new and *_by_class

Those instances are exclusively handled internally and can not be freed.

Adwaita: selection mode checks

- add missing hover asset
- have enough contrast for the outline
- :active state isn't supported in libgd/apps yet

Adwaita: keep selectmode checks square

- keep the existing 40x40px nominal size

gtk: Fix out-param annotation of gtk_theming_engine_get_style_property